-
Notifications
You must be signed in to change notification settings - Fork 374
Datasets, models and summary added to text-to-image task #378
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for contributions!!! ❤️ I left two comments, otherwise looks very good to me 🙂
The pages serve as the first stop people who would like to know about a task and maybe build a quick PoC for their business case on it so it's important to stay a bit more general on the models and the datasets side 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good to me, @osanseviero can you quickly check? I don't think we have a canonical dataset for the task BTW (not very sure)
I found two image-to-text ones which can be used for the task imo, WDYT:
https://huggingface.co/datasets/red_caps
https://huggingface.co/datasets/conceptual_captions
I'll be happy to merge after the key-space and indent errors are fixed 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great contribution, thanks! 🔥
As @merveenoyan mentioned, a couple of datasets could be a good fit here. Apart from the 2 already great ones Merve shared, the ones from LAION are quite used right now https://huggingface.co/laion (e.g. https://huggingface.co/datasets/laion/laion2B-en-aesthetic)
Co-authored-by: Omar Sanseviero <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm happy to merge, but can you install eslint and do CTRL+S on data.ts so it's formatted? There are indentation and key-value space related issues. Thanks a lot for working on this! 🙌🏻 Also you can remove about.md as it's empty.
I have fixed the indent and key-value space issues but i couldnt find the eslint config file. Could you give the configuration file you have used for future uses ? |
|
I will format it myself, thanks a lot for the contribution! What is your hugging face Hub handle so I can add your name to the doc later? 🙂 @oguzakif |
|
Did you mean my hugging face username ? https://huggingface.co/oguzakif
|
Summary